Skip to content

fix(retry): preserve sub-second retry delays#432

Merged
maxlambrecht merged 1 commit into
spiffe:mainfrom
maxlambrecht:fix/retry-duration-precision
Jul 23, 2026
Merged

fix(retry): preserve sub-second retry delays#432
maxlambrecht merged 1 commit into
spiffe:mainfrom
maxlambrecht:fix/retry-duration-precision

Conversation

@maxlambrecht

Copy link
Copy Markdown
Member

What

  • Schedule retry delays using millisecond precision instead of whole seconds.
  • Add deterministic coverage for a sub-second retry delay.

Why

RetryHandler.scheduleRetry used Duration.getSeconds(), which truncated delays below one second to zero and caused immediate retries.

How tested

  • ./gradlew :java-spiffe-core:test --tests io.spiffe.workloadapi.retry.RetryHandlerTest
  • ./gradlew :java-spiffe-core:test --tests 'io.spiffe.workloadapi.retry.*' --tests 'io.spiffe.workloadapi.DefaultWorkloadApiClientRetryableErrorTest'

@maxlambrecht
maxlambrecht requested a review from rturner3 as a code owner May 9, 2026 19:32
@maxlambrecht
maxlambrecht force-pushed the fix/retry-duration-precision branch from 0cea5d1 to cb25587 Compare May 9, 2026 20:37
@maxlambrecht
maxlambrecht force-pushed the fix/retry-duration-precision branch from cb25587 to 0945f6a Compare July 18, 2026 20:36
@maxlambrecht
maxlambrecht force-pushed the fix/retry-duration-precision branch from b850f95 to 572cc1c Compare July 23, 2026 15:30
Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
@maxlambrecht
maxlambrecht force-pushed the fix/retry-duration-precision branch from 572cc1c to 82ab49b Compare July 23, 2026 15:40
@maxlambrecht
maxlambrecht merged commit 32f16ae into spiffe:main Jul 23, 2026
8 checks passed
@maxlambrecht
maxlambrecht deleted the fix/retry-duration-precision branch July 23, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants